Skip to content

added changes to fix affected project search in vulnerabilities page#1944

Merged
nscuro merged 6 commits intomainfrom
fix/affected-projects-search-filter
Apr 16, 2026
Merged

added changes to fix affected project search in vulnerabilities page#1944
nscuro merged 6 commits intomainfrom
fix/affected-projects-search-filter

Conversation

@mehab
Copy link
Copy Markdown
Collaborator

@mehab mehab commented Mar 27, 2026

Description

This PR fixes the affected project search on the vulnerability detail page. Currently this search does not refresh if project name is provided in the search bar

Addressed Issue

If the number of affected projects is too many for a vulnerability, the search box should help us search specific project as needed. Currently even if we type something in the search box, the list does not get refreshed.

Additional Details

Checklist

  • I have read and understand the contributing guidelines
  • This PR fixes a defect, and I have provided tests to verify that the fix is effective
  • This PR implements an enhancement, and I have provided tests to verify that it works as intended
  • This PR introduces changes to the database model, and I have updated the migration changelog accordingly
  • This PR introduces new or alters existing behavior, and I have updated the documentation accordingly

@mehab mehab requested a review from nscuro March 27, 2026 15:17
@mehab mehab added the defect Something isn't working label Mar 27, 2026
Signed-off-by: Meha Bhargava <[email protected]>
@mehab mehab requested a review from sahibamittal March 30, 2026 12:48
@nscuro nscuro added this to the 5.7.0 milestone Mar 30, 2026
Signed-off-by: Meha Bhargava <[email protected]>
@codacy-production
Copy link
Copy Markdown

codacy-production bot commented Mar 31, 2026

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 0 complexity

Metric Results
Complexity 0

View in Codacy

🟢 Coverage 100.00% diff coverage

Metric Results
Coverage variation Report missing for 69f6ea01
Diff coverage 100.00% diff coverage (70.00%)

View coverage diff in Codacy

Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (69f6ea0) Report Missing Report Missing Report Missing
Head commit (eb96c45) 42867 36152 84.34%

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#1944) 5 5 100.00%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

1 Codacy didn't receive coverage data for the commit, or there was an error processing the received data. Check your integration for errors and validate that your coverage setup is correct.

TIP This summary will be updated as you push new changes. Give us feedback

Copy link
Copy Markdown
Member

@nscuro nscuro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's get the API docs cleaned up, otherwise looks good.

@mehab mehab requested a review from nscuro April 16, 2026 14:38
nscuro
nscuro previously approved these changes Apr 16, 2026
@nscuro
Copy link
Copy Markdown
Member

nscuro commented Apr 16, 2026

@mehab The new tests are failing, I think you need to add initializeWithPermissions so the requests are authorized correctly.

Signed-off-by: Meha Bhargava <[email protected]>
@nscuro nscuro merged commit 6cfe9eb into main Apr 16, 2026
9 checks passed
@nscuro nscuro deleted the fix/affected-projects-search-filter branch April 16, 2026 18:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

defect Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants